Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[새기능] 어드민 분석 기능 #99

Merged
merged 62 commits into from
Jul 9, 2024
Merged

[새기능] 어드민 분석 기능 #99

merged 62 commits into from
Jul 9, 2024

Conversation

cabbage16
Copy link
Member

🎫 관련 이슈

close #


📄 개요

어드민이 지원자 수, 성적 분포, 지원 전형 비율, 출신 학교 현황을 볼 수 있어야 합니다.


🔨 작업 내용

  • 전형별 지원자 수 API를 만들었습니다.
  • 성적 분포 API를 만들었습니다.
  • 전형별 성비 API를 만들었습니다.
  • 출신 학교 현황 API를 만들었습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

아직 테스트가 모두 완료되지 않았지만 마루 발표가 갑작스럽게 앞당겨져서 우선 병합을 하고 나중에 코드 리뷰를 받고 싶습니다. 그리고 제가 도중에

- 분석페이지에서 지원자의 출신 학교의 도로명 주소를 표기해야 하는데, 도로명주소가 없어서 컬럼을 추가했습니다.
- 테스트코드에서도 school_address 컬럼추가를 반영했습니다
- 어드민이 전형별 지원자수를 조회할 수 있습니다
- 어드민이 전형별 학생들의 최고점, 최하점, 평균을 조회할 수 있어요
- 어드민이 전형별 지원자들의 성별분포를 파악할 수 있어요.
- 어드민이 지원자들의 출신 학교 정보를 조회할 수 있어요.
- debug 로그가 너무 길어서 보이지 않도록 옵션을 false로 바꿨습니다.
- 분석페이지에서 지원자의 출신 학교의 도로명 주소를 표기해야 하는데, 도로명주소가 없어서 컬럼을 추가했습니다.
- 테스트코드에서도 school_address 컬럼추가를 반영했습니다
- 어드민이 전형별 지원자수를 조회할 수 있습니다
- 어드민이 전형별 학생들의 최고점, 최하점, 평균을 조회할 수 있어요
- 어드민이 전형별 지원자들의 성별분포를 파악할 수 있어요.
- 어드민이 지원자들의 출신 학교 정보를 조회할 수 있어요.
- debug 로그가 너무 길어서 보이지 않도록 옵션을 false로 바꿨습니다.
- 원서 조회시 schoolAddress가 보이지 않는 오류를 수정했어요.
- 지원자 수가 없는 전형은 성적 분포가 나오지 않는 것에서 0으로 나오도록 변경했습니다.
- 분석 페이지 컨트롤러 테스트
- 분석 페이지 문서 작성
- 학교를 검색할 때 학교의 도로명 주소가 함께 나오도록 수정함
- HTTP GET 메서드에서는 RequestBody를 권장하지 않는다는것을 알아서 Parameter로 받도록 변경했습니다.
- @ModelAttribute의 작동 방식이 Setter를 이용해 값을 주입하는것이어서 request dto에 setter를 추가했습니다.
- 특별 전형으로 조회했을 때 마이스터 인재전형과 정원 외 전형이 나오지 않는 오류를 해결했습니다
- 특례입학 대상자를 조회했을 때 특별전형의 정원 외 전형이 나오는 오류를 해결했습니다.
국가보훈자녀와 특례입학이 똑같이 정원 외 전형 카테고리만 있고 국가보훈자녀가 특별 전형 카테고리에 들어있지 않아 코드가 좀 더러워졌습니다...
- 분석페이지에서 지원자의 출신 학교의 도로명 주소를 표기해야 하는데, 도로명주소가 없어서 컬럼을 추가했습니다.
- 테스트코드에서도 school_address 컬럼추가를 반영했습니다
- 어드민이 전형별 지원자수를 조회할 수 있습니다
- 어드민이 전형별 학생들의 최고점, 최하점, 평균을 조회할 수 있어요
- 어드민이 전형별 지원자들의 성별분포를 파악할 수 있어요.
- 학교를 검색할 때 학교의 도로명 주소가 함께 나오도록 수정함
- HTTP GET 메서드에서는 RequestBody를 권장하지 않는다는것을 알아서 Parameter로 받도록 변경했습니다.
- @ModelAttribute의 작동 방식이 Setter를 이용해 값을 주입하는것이어서 request dto에 setter를 추가했습니다.
- 특별 전형으로 조회했을 때 마이스터 인재전형과 정원 외 전형이 나오지 않는 오류를 해결했습니다
- 특례입학 대상자를 조회했을 때 특별전형의 정원 외 전형이 나오는 오류를 해결했습니다.
국가보훈자녀와 특례입학이 똑같이 정원 외 전형 카테고리만 있고 국가보훈자녀가 특별 전형 카테고리에 들어있지 않아 코드가 좀 더러워졌습니다...
- 분석페이지에서 지원자의 출신 학교의 도로명 주소를 표기해야 하는데, 도로명주소가 없어서 컬럼을 추가했습니다.
- 테스트코드에서도 school_address 컬럼추가를 반영했습니다
- 어드민이 전형별 지원자수를 조회할 수 있습니다
- 어드민이 전형별 학생들의 최고점, 최하점, 평균을 조회할 수 있어요
- 어드민이 전형별 지원자들의 성별분포를 파악할 수 있어요.
- 어드민이 지원자들의 출신 학교 정보를 조회할 수 있어요.
- debug 로그가 너무 길어서 보이지 않도록 옵션을 false로 바꿨습니다.
- 원서 조회시 schoolAddress가 보이지 않는 오류를 수정했어요.
- 지원자 수가 없는 전형은 성적 분포가 나오지 않는 것에서 0으로 나오도록 변경했습니다.
- 분석 페이지 컨트롤러 테스트
- 분석 페이지 문서 작성
- 학교를 검색할 때 학교의 도로명 주소가 함께 나오도록 수정함
- HTTP GET 메서드에서는 RequestBody를 권장하지 않는다는것을 알아서 Parameter로 받도록 변경했습니다.
- @ModelAttribute의 작동 방식이 Setter를 이용해 값을 주입하는것이어서 request dto에 setter를 추가했습니다.
- 특별 전형으로 조회했을 때 마이스터 인재전형과 정원 외 전형이 나오지 않는 오류를 해결했습니다
- 특례입학 대상자를 조회했을 때 특별전형의 정원 외 전형이 나오는 오류를 해결했습니다.
국가보훈자녀와 특례입학이 똑같이 정원 외 전형 카테고리만 있고 국가보훈자녀가 특별 전형 카테고리에 들어있지 않아 코드가 좀 더러워졌습니다...
QueryNumberOfApplicantsUseCaseTest를 작성했습니다.
@cabbage16 cabbage16 requested a review from a team as a code owner July 9, 2024 00:56
@cabbage16 cabbage16 linked an issue Jul 9, 2024 that may be closed by this pull request
6 tasks
Copy link

github-actions bot commented Jul 9, 2024

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

Copy link

sonarcloud bot commented Jul 9, 2024

@cabbage16 cabbage16 merged commit d8261e9 into develop Jul 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[새기능] 어드민 분석 기능
1 participant